Skip to content

Remove animations when it's disabled in system settings#222

Merged
JingMatrix merged 5 commits intoJingMatrix:masterfrom
stevenxxiu:feat/remove-animation
Mar 14, 2025
Merged

Remove animations when it's disabled in system settings#222
JingMatrix merged 5 commits intoJingMatrix:masterfrom
stevenxxiu:feat/remove-animation

Conversation

@stevenxxiu
Copy link
Copy Markdown
Contributor

Closes #185.

@stevenxxiu stevenxxiu force-pushed the feat/remove-animation branch 2 times, most recently from 825b1be to 7dda6e1 Compare March 9, 2025 14:32
@stevenxxiu stevenxxiu force-pushed the feat/remove-animation branch from 7dda6e1 to 4e6bc6d Compare March 11, 2025 08:42
@stevenxxiu stevenxxiu force-pushed the feat/remove-animation branch from 4e6bc6d to f5a5cff Compare March 11, 2025 09:26
To navigate without animations, the value for arguments of type @AnimatorRes should be -1.
@JingMatrix
Copy link
Copy Markdown
Owner

JingMatrix commented Mar 13, 2025

If possible, please consider hiding the component CircularProgressIndicator SwipeRefreshLayout.
It is inconsistent to have a progress indicator with other animations disabled.

@stevenxxiu
Copy link
Copy Markdown
Contributor Author

If possible, please consider hiding the component CircularProgressIndicator SwipeRefreshLayout. It is inconsistent to have a progress indicator with other animations disabled.

If SwipeRefreshLayout is removed, then is there a way to tell when the refresh finishes? Otherwise it could be something that's informative. I don't mind spinner loading animations for that reason.

@JingMatrix
Copy link
Copy Markdown
Owner

Okay for that.

I wonder if if the smoothScroll parameter of TabLayoutMediator really creates animations.
I cannot tell the difference from my device.

@stevenxxiu
Copy link
Copy Markdown
Contributor Author

stevenxxiu commented Mar 13, 2025

I wonder if if the smoothScroll parameter of TabLayoutMediator really creates animations. I cannot tell the difference from my device.

It should remove the sliding animation that shows up when you switch tabs in the Logs panel or in a Repository panel's module.

I tried this in an emulator before, and following your comments tried it on my real phone. It works there too.

Are you sure you disabled all animations in system settings? "Animation duration scale", "Window animation scale", "Transition animation scale".

@JingMatrix JingMatrix merged commit 90d3712 into JingMatrix:master Mar 14, 2025
1 check passed
@stevenxxiu stevenxxiu deleted the feat/remove-animation branch March 14, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Follow system wide animation settings

2 participants